Skip to content

@W-20615533 Search popup – Ask Shopping Agent banner and Embedded Messaging#3658

Merged
sf-tejas-nadkarni merged 3 commits intodevelopfrom
W-20615533-search-popup-extension
Feb 18, 2026
Merged

@W-20615533 Search popup – Ask Shopping Agent banner and Embedded Messaging#3658
sf-tejas-nadkarni merged 3 commits intodevelopfrom
W-20615533-search-popup-extension

Conversation

@sf-tejas-nadkarni
Copy link
Collaborator

@sf-tejas-nadkarni sf-tejas-nadkarni commented Feb 12, 2026 •
edited
Loading

Description

Adds an “Ask Shopping Agent” entry point in the search suggestions popup and wires it to the existing Embedded Messaging (Shopper Agent) chat. When the feature is enabled, users see a banner in the search dropdown that opens the Shopping Agent chat on click.

Screenshot 2026-02-13 at 12 23 19 PM

Types of Changes

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation update
  • Breaking change (could cause existing functionality to not work as expected)
  • Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

  • Removing a public function or component or prop
  • Adding a required argument to a function
  • Changing the data type of a function parameter or return value
  • Adding a new peer dependency to package.json

Changes

  • Search popup – Ask Assistant
    AskAssistantBanner – New banner in search suggestions (desktop and mobile) with title “Ask Shopping Agent” and short description; click opens chat and clears the search input.

  • Search component – Reads enableAgentFromSearchSuggestions from commerce agent config and passes onAskAssistantClick (calls launchChat() + clearInput()) into suggestion partials.

How to Test-Drive This PR

  • Enable commerce agent and set enableAgentFromSearchSuggestions: true in config (or use current defaults).
  • Open search and focus the search input to open the suggestions popup.
  • Confirm the “Ask Shopping Agent” banner appears (with and without other suggestions).
  • Click the banner and confirm the Embedded Messaging chat opens and the search input is cleared.

Checklists

General

  • Changes are covered by test cases
  • CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance

You must check off all items in one of the follow two lists:

  • There are no changes to UI

or...

Localization

  • Changes include a UI text update in the Retail React App (which requires translation)

@cc-prodsec
Copy link
Collaborator

cc-prodsec commented Feb 12, 2026 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@sf-tejas-nadkarni sf-tejas-nadkarni force-pushed the W-20615533-search-popup-extension branch from 3121fd8 to 7383290 Compare February 13, 2026 00:22
@sf-tejas-nadkarni sf-tejas-nadkarni marked this pull request as ready for review February 13, 2026 00:23
@sf-tejas-nadkarni sf-tejas-nadkarni requested a review from a team as a code owner February 13, 2026 00:23
@sf-tejas-nadkarni sf-tejas-nadkarni changed the title @W-20615533 search popup extension @W-20615533 Search popup – Ask Shopping Agent banner and Embedded Messaging Feb 13, 2026
@sf-tejas-nadkarni sf-tejas-nadkarni added the skip changelog Skip the "Changelog Check" GitHub Actions step even if the Changelog.md files are not updated label Feb 13, 2026
@sf-tejas-nadkarni sf-tejas-nadkarni force-pushed the W-20615533-search-popup-extension branch from f7d4c92 to 111c076 Compare February 16, 2026 22:09
shethj
shethj reviewed Feb 17, 2026
View reviewed changes
@sf-tejas-nadkarni sf-tejas-nadkarni merged commit f7847c5 into develop Feb 18, 2026
42 checks passed
@sf-tejas-nadkarni sf-tejas-nadkarni deleted the W-20615533-search-popup-extension branch February 18, 2026 05:26
sf-jhalak-maheshwari
sf-jhalak-maheshwari reviewed Feb 23, 2026
View reviewed changes
packages/template-retail-react-app/app/utils/config-utils.js
enableAgentFromFloatingButton: 'false',
enableAgentFromSearchSuggestions: 'false'
}
return getConfig().app.commerceAgent ?? defaults
Copy link
Contributor

@sf-jhalak-maheshwari sf-jhalak-maheshwari Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sf-tejas-nadkarni forgot to add this -
With ?? only the commerceAgent configs will be taken in and not the default. Just want to check if we have added all the null/undefined checks if commerceAgent doesent pass in the config keys

Copy link
Contributor

@sf-jhalak-maheshwari sf-jhalak-maheshwari Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm, we should be good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sf-jhalak-maheshwari sf-jhalak-maheshwari sf-jhalak-maheshwari left review comments

@kevinxh kevinxh kevinxh approved these changes

@shethj shethj shethj approved these changes

Assignees

No one assigned

Labels

skip changelog Skip the "Changelog Check" GitHub Actions step even if the Changelog.md files are not updated

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sf-tejas-nadkarni @cc-prodsec @kevinxh @shethj @sf-jhalak-maheshwari